Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: several typos in the documentation #929

Merged
merged 22 commits into from
Nov 16, 2023

Conversation

tudorpintea999
Copy link
Contributor

@tudorpintea999 tudorpintea999 commented Nov 11, 2023

Description of the Changes

Hello
I fixed some typos in the docs,
Hope it helps.
Br,
Tudor.


This change is Reviewable

Copy link
Collaborator

@stoobie stoobie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great comments! Thank you so much :)

Please correct the few things I mentioned, and then it's good to go.

If you prefer that I take care of it, just let me know.

@@ -70,7 +70,7 @@ When working with the CLI, you can manually set the endpoints for the gateways t
interact with Starknet, by including the following options:

`--feeder_gateway_url`:: Sets the custom endpoint for read commands.
`--gateway_url`:: Sets the custom endpoint for write commands.
`--gateway_url`:: Sets the custom endpoint for writing commands.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is incorrect. The sentence is talking about read/write commands.

components/Starknet/modules/cli/pages/starkli.adoc Outdated Show resolved Hide resolved
contributing_to_docs/doc_guidelines.adoc Outdated Show resolved Hide resolved
@tudorpintea999
Copy link
Contributor Author

hello @stoobie i didn t see the notification 3 days ago, check now please. Thanks !

@stoobie stoobie changed the base branch from dev to tudorpintea999/fix-typo November 16, 2023 07:48
@stoobie stoobie merged commit 2553f91 into starknet-io:tudorpintea999/fix-typo Nov 16, 2023
1 of 2 checks passed
@stoobie
Copy link
Collaborator

stoobie commented Nov 16, 2023

Merged to new branch to generate preview before merging to dev.

stoobie added a commit that referenced this pull request Nov 17, 2023
* Fix: several typos in the documentation (#929)

* Update cairo-and-sierra.adoc

* Update contract-abi.adoc

* Update contract-address.adoc

* Update contract-classes.adoc

* Update contract-storage.adoc

* Update hash-functions.adoc

* Update starknet-state.adoc

* Update starkli.adoc

* Update declare_a_smart_contract.adoc

* Update juno_versions.adoc

* Update pathfinder_versions.adoc

* fix typo doc_guidelines.adoc

* fix contract-abi.adoc

Co-authored-by: Steve Goodman <[email protected]>

* fix starkli.adoc

Co-authored-by: Steve Goodman <[email protected]>

* fix declare_a_smart_contract.adoc

Co-authored-by: Steve Goodman <[email protected]>

* fix doc_guidelines.adoc

Co-authored-by: Steve Goodman <[email protected]>

* Update starkli.adoc

* Update cairo-and-sierra.adoc

* Update components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/contract-abi.adoc

---------

Co-authored-by: Steve Goodman <[email protected]>

* Apply suggestions from code review

---------

Co-authored-by: iwantanode <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants